feat: デフォルトでフォローするユーザーを指定できるように #14769
Conversation
github-actions
bot
added
packages/frontend
|
このPRによるapi.jsonの差分 差分はこちら--- base
+++ head
@@ -175,6 +175,18 @@
"type": "string"
}
},
+ "defaultFollowedUsers": {
+ "type": "array",
+ "items": {
+ "type": "string"
+ }
+ },
+ "forciblyFollowedUsers": {
+ "type": "array",
+ "items": {
+ "type": "string"
+ }
+ },
"hiddenTags": {
"type": "array",
"items": {
@@ -609,6 +621,8 @@
"translatorAvailable",
"mediaSilencedHosts",
"pinnedUsers",
+ "defaultFollowedUsers",
+ "forciblyFollowedUsers",
"hiddenTags",
"blockedHosts",
"sensitiveWords",
@@ -13817,6 +13831,26 @@
"type": "string"
}
},
+ "defaultFollowedUsers": {
+ "type": [
+ "array",
+ "null"
+ ],
+ "items": {
+ "type": "string",
+ "format": "misskey:id"
+ }
+ },
+ "forciblyFollowedUsers": {
+ "type": [
+ "array",
+ "null"
+ ],
+ "items": {
+ "type": "string",
+ "format": "misskey:id"
+ }
+ },
"hiddenTags": {
"type": [
"array",
@@ -14416,6 +14450,15 @@
"$ref": "#/components/schemas/Error"
},
"examples": {
+ "FOLLOWED_USER_DUPLICATED": {
+ "value": {
+ "error": {
+ "message": "Some items in \"defaultFollowedUsers\" and \"forciblyFollowedUsers\" are duplicated.",
+ "code": "FOLLOWED_USER_DUPLICATED",
+ "id": "bcf088ec-fec5-42d0-8b9e-16d3b4797a4d"
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -20461,6 +20504,16 @@
}
}
},
+ "CANNOT_BLOCK_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot block that user due to server policy.",
+ "code": "CANNOT_BLOCK_DUE_TO_SERVER_POLICY",
+ "id": "e2f04d25-0d94-4ac3-a4d8-ba401062741b",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -36766,6 +36819,16 @@
}
}
},
+ "CANNOT_UNFOLLOW_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot unfollow that user due to server policy.",
+ "code": "CANNOT_UNFOLLOW_DUE_TO_SERVER_POLICY",
+ "id": "19f25f61-0141-4683-99dc-217a88d633cb",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -54579,6 +54642,16 @@
}
}
},
+ "CANNOT_MUTE_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot mute that user due to server policy.",
+ "code": "CANNOT_MUTE_DUE_TO_SERVER_POLICY",
+ "id": "15273a89-374d-49fa-8df6-8bb3feeea455",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": {
@@ -55116,6 +55189,16 @@
}
}
},
+ "CANNOT_MUTE_DUE_TO_SERVER_POLICY": {
+ "value": {
+ "error": {
+ "message": "You cannot mute that user due to server policy.",
+ "code": "CANNOT_MUTE_DUE_TO_SERVER_POLICY",
+ "id": "15273a89-374d-49fa-8df6-8bb3feeea455",
+ "httpStatusCode": 403
+ }
+ }
+ },
"INVALID_PARAM": {
"value": {
"error": { |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14769 +/- ##
===========================================
+ Coverage 39.98% 41.59% +1.61%
===========================================
Files 1563 1567 +4
Lines 197737 203794 +6057
Branches 3644 3666 +22
===========================================
+ Hits 79056 84766 +5710
- Misses 118077 118422 +345
- Partials 604 606 +2 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
|
/preview |
|
フォロー解除できないユーザーのほうのプロパティ名が微妙なので募集中 |
|
permanentFollowedはpermientが形容詞だけどfollowedは名詞じゃないので品詞的におかしくてpermanentlyのほうが適切だとは思います。そもそも単語選択レベルでもっといいのあるかもだけど |
|
forceとか |
|
fixedとか |
の両方を取り入れて |
そうした |
|
ピン留めユーザーの設定はacct形式での入力を求めるのに対して、（処理を簡略化する都合上）こちらではユーザーIDを入力させるようにしてあるけど、改めたほうがいいかしら（管理者画面なので「IDを入力せよ」と明記するだけで良い説はある (すでにそれはやってある)） |
これユーザーにとって不便になる影響の方が大きいんじゃないかしら |
だからサーバーの運用に合わせて管理者がうまいこと使い分ければ良さそう（ユーザーでフォロー解除できる方も別個の設定として存在しているので） |
|
ほむん |
お知らせ管理画面周りみたく、別途MkInfoか何かで「フォロー解除できない方の設定は最低限のアカウントにすることをおすすめします」とかを書くといいのかもしれない |
|
インポート機能使えばミュートやブロック不可制限を回避できそうな予感 |
あーそっちは塞いでなかったわね |
これどうだろう |
塞いだ |
|
UserMutingService.muteとかに当該ロジックあった方が自然だし漏れが無くなるけどそうしなかった理由はあるかしら |
あーそっちに入れるほうがいいか（別の内部的な処理とかち合うとややこしそうかなと思ったので） |
|
コンフリクト解消 |
|
コンフリクト解消 |
|
翻訳キー・マイグレーションキーの置き換えが不十分だった |
多分直った気がする |
What
Why
Fix #12451
Additional info (optional)
フォロー解除できるユーザーと解除できないユーザーのフィールドを分けずに、別途スイッチでフォロー解除できるようにするかどうかを選べるようにすることも考えたけど、両方のフィールドを使う運用も考えられた（推奨するユーザーも必須なユーザーもはじめからフォローしておいてもらうような運用）ので両方別で残している
Checklist